a11y tests: Use the new AtkValue interface

The old interface is deprecated, and we just implemented the
new one, so lets test it.

GtkDialog: Reduce a g_warning to a g_message

Otherwise, this breaks our testsuite.

GtkSpinButton: Implement new AtkValue interface

The AtkValue interface has been replaced in ATK 2.12.
Implement the new one in addition to the old one.

GtkScaleButton: Implement new AtkValue interface

The AtkValue interface has been replaced in ATK 2.12.
Implement the new one in addition to the old one.

GtkRange: Implement new AtkValue interface

The AtkValue interface has been replaced in ATK 2.12.
Implement the new one in addition to the old one.

GtkProgressBar: Implement new AtkValue interface

The AtkValue interface has been replaced in ATK 2.12.
Implement the new one in addition to the old one.

GtkPaned: Implement new AtkValue interface

The AtkValue interface has been replaced in ATK 2.12.
Implement the new one in addition to the old one.

GtkLevelBar: Implement new AtkValue interface

The AtkValue interface has been replaced in ATK 2.12.
Implement the new one in addition to the old one.

GtkWidget: Mark deprecated properties as such

The ::style, ::margin-left and ::margin-right properties are all
deprecated.

Drop unneeded deprecation suppression

Removing this did not produce any deprecation warnings.

queryimmodules: Localize suppressed deprecation

Drop the blanket GDK_DISABLE_DEPRECATION_WARNINGS and avoid
deprecated API as much as possible.

GtkColorButton: Reduce internal use of deprecated API

Drop the blanket GDK_DISABLE_DEPRECATION_WARNINGS and avoid
deprecated API as much as possible.

GtkColorButton: Update deprecation warnings

It is not great to have deprecation warnings point to another
deprecated function. Update them to point to the eventual
replacement.

GtkDialog: Warn about lack of transient parents

We want make it mandatory for dialogs to have transient parents,
eventually. This is a first step in that direction.

teststack: don't enumerate transitions manually

Thats what introspection is for.

GtkStack: Avoid compiler warnings

gcc warns about unhandled enum values in these switch statements,
so add an empty default: case to placate the compiler.

stack: Add counterparts of OVER_UP_DOWN transition

The GtkStackTransitionType enum now has OVER_DOWN_UP, OVER_LEFT_RIGHT,
and OVER_RIGHT_LEFT values to complement OVER_UP_DOWN.

https://bugzilla.gnome.org/show_bug.cgi?id=726676

stack: More readability when adding transitions

This small refactor makes the code more readable when adding more
transition types that have left, right, up, and down variations.

It adds inline boolean functions to tell information about transition
types (avoiding long if clauses) and changes long chains of "else if
(transition_type == ...)" into switch statements. Both are only likely
to get longer as more transition types are added.

https://bugzilla.gnome.org/show_bug.cgi?id=726676

overlay: don't have a custom draw implementation

We can't think with Benjamin of a reason this should be needed, so let's
just remove it.

themingengine: only round outline corners after shrinking

Ensures the correctness of the applied radius.

GtkBuilder: added template parent type check.

GtkFlowBox: fixed NULL dereferencing segfaults

GtkMenu: fixed more NULL dereferencing segfault

GtkGrid: gtk_grid_get_size_for_size() do not dereference a NULL pointer.

GtkBuilder: improved parsing error report for invalid properties and signals.

Added GTK_BUILDER_ERROR_INVALID_PROPERTY and GTK_BUILDER_ERROR_INVALID_SIGNAL
error codes

ObjectInfo: Use a GType instead of a char * for the class name.
PropertyInfo: Use a GParamSpec instead of a char * for the property name.
SignalInfo: Use signal id and detail quark instead of a detailed signal name string.

This not only save us a few malloc in each case but lets us simplify the code
and report unknown properties and signals as a parsing error instead of just
printing a warning.

GtkBuilder: fixed a few minor memory leaks.

Fixed memory leaks in parse_object(), parse_template() and parse_signal() functions.
Parameters value where strduped before the last posible return and not freed.

W32: override selected button style to remove colored background

https://bugzilla.gnome.org/show_bug.cgi?id=729353

W32: Add a theme for .app-notification

This is copied from Adwaita. Looks sufficiently ok.

https://bugzilla.gnome.org/show_bug.cgi?id=729346

listbox: fix a segfault

Don't dereference a NULL pointer.

combobox: fix a segfault

Don't dereference a NULL pointer.

themingengine: use outlines to render focus rings

Instead of the old non-CSS implementation. This allows us to get rid of
the very last piece of custom theming in Adwaita.

css: add properties for outline-radius

Both a shorthand and individual properties. This is not officially part
of the CSS standard, but there's precedence for it at least in Mozilla:
https://developer.mozilla.org/en-US/docs/Web/CSS/-moz-outline-radius

treeview: respect focus-padding when rendering focus

Adwaita had a hack to work this around, but now that we're moving focus
ring rendering into GTK, we can just fix the treeview instead.

treeview: Always call size request funcs with valid out pointers

Not doing so causes crashes since
a158a2aa48c8023f99963642cc2657bff207b82NULLd

revealer: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

popover: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

menuitem: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

menu: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

levelbar: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

eventbox: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

listbox: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

frame: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

headerbar: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

infobar: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

grid: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

scrolledwindow: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

switch: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

toolbar: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

treeview: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

viewport: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

viewport: Implement hfw properly

Previously, the code would fall back on GtkBin's hfw implementation
which didn't account for padding and borders.

viewport: Use a convenience function

That function looks longer than not using it, but we'll need it in the
next commit.

viewport: Request 2x border-width

It's nice that bugs get pointed out in code comments. Fixing them would
probably have been preferrable though.

widget: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

flowbox: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

combobox: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

button: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

alignment: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

arrow: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

spinner: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

modules: remove the pixbuf engine directory

This is not used anywhere, and is commented out of the build because it
doesn't compile. Time for some spring cleaning...

Updated POTFILES.skip

GtkSearchBar: fix the template

There was a mismatch between GtkSearchBar being derived from GtkBin
and the template claiming the parent to be GtkBox.

widget-factory: populate the second page more

Add back a menubar example that was lost a while ago, and also
include a searchbar and an infobar.

widget-factory: rearrange page 2 a bit

reftests: Add test for latest commit

Check that the label gets linebroken properly.

window: Subtract shadow from passed in width for height

... and height for width in size requests. Fixes mislayout in
control-center universal access panel.

widgets: Remove (GtkTickCallback) casts

Instead, make the functions conform to the prototype, so that casting
isn't needed.

GtkMenuSectionBox: Don't show separators for nested subsections

...unless they are labeled.

To implement this, keep track of the nesting depth of section boxes,
starting from 0 for the toplevel and the submenu ones, and only
insert unlabeled separators if the depth is at most 1.

GtkMenuSectionBox: Always show separators for labeled sections

...unless they are empty.

Add a comment with rules for separators

Added template test case to builder test.

GtkMenuSectionBox: don't leak

This wasn't fully fleshed out yet.

widget-factory: Add a horizontal-buttons example

Sadly, we don't have good icons for payment methods...

treeview: Fix prelight redraw when adjustment changes

When the adjustment changes (due to e.g. a mouse wheel scroll) we update
the prelight. The part that un-prelights the previous prelight was
broken by the the pixel cache, as it called update_prelight in the
middle of the scrolling operation, where the windows were moved
but the tree_view->priv->dy was not changed to the new value. This
caused the updates to the pixel cache to go to the wrong place.

We fix this by fully doing the scroll before we update_prelight().

https://bugzilla.gnome.org/show_bug.cgi?id=728284

3.13.1

Update output of a11y tests

These are all affected by the addition of the action_box in
GtkDialog or by the switch to popovers in GtkMenuButton.

GtkMenuShell: We want separators in submenus

This was probably an oversight when adding the merge_separators
parameter. Without this change, the testsuite/gtk/gtkmenu test
fails.

GtkMenuShell: Don't use a non-existing property

GtkMenuTrackerItem::visible was removed a few commits ago.
It is not necessary to bind visible anyway, since the menu
tracker will insert and remove items as their visibility
changes.

testmenubutton: Add a toggle for use-popover

gdkwindow: ignore shape on client-side windows when recomputing visible regions

Rendering doesn't do much about clipping drawing operations to the window shape,
although invalidation applies the shape to every window, leaving possibly trails
of "overrendered" content. So ensure the shape portions get invalidated too when
the window is moved/resized.

https://bugzilla.gnome.org/show_bug.cgi?id=729095

popover: Always make room for the tail on all sides on size request

Now that popovers may snap to any side with enough space, make enough
room on every side when requesting size, so that there's no w/h differences
at the time of setting the child allocation.

https://bugzilla.gnome.org/show_bug.cgi?id=729097

GtkRecentChooserMenu: Set current uri before activating an item.

When activating an item using mnemonics GtkRecentChooserMenu does not select a
item before calling the item-activated signal thus
gtk_recent_chooser_get_current_uri() always return the last selected item
instead of the activated one.

Fixes Bug 495105 "Open recent file keyboard shortcuts do not work correctly"

Raleigh: fix a typo

GtkDialog: protect against show_all

The real reason for the problem that the just reverted change
was supposed to address is that testfilechooser uses show_all
to tease out places where composite dialogs don't properly
protect their internals against unintended showing. Well,
lets do that, then.

Revert "GtkFileChooserDialog: Avoid a bottom border in the dialog"

This reverts commit d871105fa8fd20f9be0d07953ee2dd7d90b4c84f.

Updated Slovenian translation

Updated Slovenian translation

Document iconic section support

https://bugzilla.gnome.org/show_bug.cgi?id=727477

Add an example of iconic rendering

testpopover now shows several examples of icon buttons.

https://bugzilla.gnome.org/show_bug.cgi?id=727477

GtkMenuSectionBox: add support for iconic sections

Sections that have the display-hint attribute set to
"horizontal-boxes" will be rendered as a row of icon
buttons, instead of menuitem-like.

GtkMenuTrackerItem: Add support for display-hint

This commit adds a getter for the display-hint attribute.

GtkModelButton: Add an iconic mode

In iconic mode, model buttons will be styled like regular icon
buttons, preferring to show only the icon if one is set, falling
back to showing the label.

https://bugzilla.gnome.org/show_bug.cgi?id=727477

GtkMenuTrackerItem: Add support for verb-icons

When rendering iconic sections, we want to use icons for verbs,
and we want to differentiate these in the menu model, to keep
the icon attribute reserved for nouns.

https://bugzilla.gnome.org/show_bug.cgi?id=727477

GtkPopover: new approach to menu model binding

Instead of using GtkMenuTracker to flatten the sections into a single
linear menu, handle the sections ourselves by nesting boxes.

Each section gets an inner and outer box.  The inner box numbers its
children in the way that the tracker instructs.  The outer box
containes the inner box and the separator, if appropriate.

Having the two separate boxes will allow us to change the orientation of
the inner box if we want to pack widgets horizontally within a section.

GtkMenuTracker: add 'merge_sections' flag

Add the possibility of a GtkMenuTracker that performs no section
merging.  Instead, it will report an item in the form of a separator for
subsections.  It is then possible to get a separate tracker for the
subsection contents by using gtk_menu_tracker_new_for_item_link().

GtkMenuTracker: don't specialise "submenu" link

We have some API in GtkMenuTracker and GtkMenuTrackerItem that is
specifically designed to deal with submenus.

Generalise these APIs to take a 'link_name' parameter that we always
give as G_MENU_SUBMENU for now.  In the future, this will allow creating
trackers for other types of links, such as sections.

GtkMenuTracker: make "is-visible" a property

Make this a property just like all of the other things and make the APIs
for accessing it non-private (but add a note that they are not intended
to be used).

GtkMenuTracker: remove "visible" property

This is in conflict with the "is-visible" pseudo-property which is about
to be promoted to being an actual property.

W32: Update the css to match the .modelbutton->.button.menuitem transition